fix: add pino and rxjs as peer depencies #1004 #2103
Conversation
|
@iamolegga hi! any chance you get some time to check and validate this PR? :) |
|
Hi, thanks for reminding me. I am currently too busy at work, but I am planning to get back to this (and other PRs) during the holidays, as it takes some time to carefully review all the version ranges to not to break things for a huge amount of dependants. There are other related PRs that could be merged too. Thanks for understanding 🙏 |
|
@ch3ric hi! Could you rebase this PR on top of the fresh master, sort lines alphabetically, and squash commits? |
- pino-http 6.4.0 requires pino ^7.5.0: https://github.com/pinojs/pino-http/blob/v6.4.0/package.json - pino-http 8 & 9 require pino ^8: https://github.com/pinojs/pino-http/blob/v8.6.1/package.json & https://github.com/pinojs/pino-http/blob/v9.0.0/package.json - pino-http 10 requires pino ^9: https://github.com/pinojs/pino-http/blob/v10.3.0/package.json - nestjs/common 8 to 11 require rxjs ^7.1.0
Hello @iamolegga |
As discussed there: #1004 (comment)